Skip to content

Update theme-language-server-common for new parser#1256

Merged
karreiro merged 3 commits into
plhp/4-resilientfrom
plhp/5-language-server
Jul 17, 2026
Merged

Update theme-language-server-common for new parser#1256
karreiro merged 3 commits into
plhp/4-resilientfrom
plhp/5-language-server

Conversation

@karreiro

@karreiro karreiro commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Updates theme-language-server-common to consume the new hand-written parser.

Part 5 of 7 in the stacked series, based on PR 4.

@karreiro
karreiro force-pushed the plhp/5-language-server branch from 74016f1 to 83fb0be Compare July 16, 2026 08:56
@karreiro
karreiro force-pushed the plhp/5-language-server branch 2 times, most recently from 2919cfb to 77d8850 Compare July 16, 2026 20:04
@karreiro
karreiro force-pushed the plhp/5-language-server branch 2 times, most recently from ac7b49c to b94059a Compare July 17, 2026 10:34
Restores completion behavior lost in the NEW liquid parser so it matches
OLD across empty/unclosed slots, pipe/filter routing, literal
type-narrowing, liquid-inner-statement, named-filter-arg value slots, and
abutting end-tag cases (A-G, R1, R2, R3).

Fix branch-tag completion parity when abutting end tag

Extends the abutting-endtag recovery to LiquidBranch (elsif/when); the R3 fix covered LiquidTag/ForMarkup only.
@karreiro
karreiro force-pushed the plhp/5-language-server branch from b94059a to 6032fad Compare July 17, 2026 11:07
…r new parser (#1257)

* Update theme-check-common, prettier-plugin-liquid, and theme-graph for new parser

* Restore baseline fixture output for 10 parser and printer regressions

Parser:
- paginate tolerant argument parsing
- comment/doc depth-balance
- conditional comments
- filter comma leniency
- curly quotes
- markdown script
- nested same-name HTML tags
- inline # indentation

Printer:
- liquid-doc trims
- tablerow branched-body

* Add changeset for hand-written liquid-html-parser (#1258)

* Remove unnecessary change
@karreiro
karreiro marked this pull request as ready for review July 17, 2026 12:08
@karreiro
karreiro requested a review from a team as a code owner July 17, 2026 12:08
@karreiro
karreiro merged commit 8c3c639 into plhp/4-resilient Jul 17, 2026
8 checks passed
@karreiro
karreiro deleted the plhp/5-language-server branch July 17, 2026 12:19
karreiro added a commit that referenced this pull request Jul 17, 2026
* Add resilient liquid-html parsing

* rename to tolerant

* Update theme-language-server-common for new parser (#1256)

* Update theme-language-server-common for new parser

* Fix NEW parser completion parity regressions

Restores completion behavior lost in the NEW liquid parser so it matches
OLD across empty/unclosed slots, pipe/filter routing, literal
type-narrowing, liquid-inner-statement, named-filter-arg value slots, and
abutting end-tag cases (A-G, R1, R2, R3).

Fix branch-tag completion parity when abutting end tag

Extends the abutting-endtag recovery to LiquidBranch (elsif/when); the R3 fix covered LiquidTag/ForMarkup only.

* Update theme-check-common, prettier-plugin-liquid, and theme-graph for new parser (#1257)

* Update theme-check-common, prettier-plugin-liquid, and theme-graph for new parser

* Restore baseline fixture output for 10 parser and printer regressions

Parser:
- paginate tolerant argument parsing
- comment/doc depth-balance
- conditional comments
- filter comma leniency
- curly quotes
- markdown script
- nested same-name HTML tags
- inline # indentation

Printer:
- liquid-doc trims
- tablerow branched-body

* Add changeset for hand-written liquid-html-parser (#1258)

* Remove unnecessary change
karreiro added a commit that referenced this pull request Jul 17, 2026
* Adapt liquid-html-parser to theme-tools source model

* Add resilient liquid-html parsing (#1255)

* Add resilient liquid-html parsing

* rename to tolerant

* Update theme-language-server-common for new parser (#1256)

* Update theme-language-server-common for new parser

* Fix NEW parser completion parity regressions

Restores completion behavior lost in the NEW liquid parser so it matches
OLD across empty/unclosed slots, pipe/filter routing, literal
type-narrowing, liquid-inner-statement, named-filter-arg value slots, and
abutting end-tag cases (A-G, R1, R2, R3).

Fix branch-tag completion parity when abutting end tag

Extends the abutting-endtag recovery to LiquidBranch (elsif/when); the R3 fix covered LiquidTag/ForMarkup only.

* Update theme-check-common, prettier-plugin-liquid, and theme-graph for new parser (#1257)

* Update theme-check-common, prettier-plugin-liquid, and theme-graph for new parser

* Restore baseline fixture output for 10 parser and printer regressions

Parser:
- paginate tolerant argument parsing
- comment/doc depth-balance
- conditional comments
- filter comma leniency
- curly quotes
- markdown script
- nested same-name HTML tags
- inline # indentation

Printer:
- liquid-doc trims
- tablerow branched-body

* Add changeset for hand-written liquid-html-parser (#1258)

* Remove unnecessary change
karreiro added a commit that referenced this pull request Jul 17, 2026
* Configure liquid-html-parser package

Assisted-By: devx/bf1b845f-ea09-4e31-9686-5c3b1cb5ebc6

* Adapt liquid-html-parser to theme-tools source model (#1254)

* Adapt liquid-html-parser to theme-tools source model

* Add resilient liquid-html parsing (#1255)

* Add resilient liquid-html parsing

* rename to tolerant

* Update theme-language-server-common for new parser (#1256)

* Update theme-language-server-common for new parser

* Fix NEW parser completion parity regressions

Restores completion behavior lost in the NEW liquid parser so it matches
OLD across empty/unclosed slots, pipe/filter routing, literal
type-narrowing, liquid-inner-statement, named-filter-arg value slots, and
abutting end-tag cases (A-G, R1, R2, R3).

Fix branch-tag completion parity when abutting end tag

Extends the abutting-endtag recovery to LiquidBranch (elsif/when); the R3 fix covered LiquidTag/ForMarkup only.

* Update theme-check-common, prettier-plugin-liquid, and theme-graph for new parser (#1257)

* Update theme-check-common, prettier-plugin-liquid, and theme-graph for new parser

* Restore baseline fixture output for 10 parser and printer regressions

Parser:
- paginate tolerant argument parsing
- comment/doc depth-balance
- conditional comments
- filter comma leniency
- curly quotes
- markdown script
- nested same-name HTML tags
- inline # indentation

Printer:
- liquid-doc trims
- tablerow branched-body

* Add changeset for hand-written liquid-html-parser (#1258)

* Remove unnecessary change
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant